Skip to content

chore(pipeline): split Build and Test stage#1071

Merged
MarkEWaite merged 1 commit intojenkinsci:masterfrom
lemeurherve:split-build-and-test-stages
Oct 5, 2025
Merged

chore(pipeline): split Build and Test stage#1071
MarkEWaite merged 1 commit intojenkinsci:masterfrom
lemeurherve:split-build-and-test-stages

Conversation

@lemeurherve
Copy link
Copy Markdown
Member

@lemeurherve lemeurherve commented Oct 5, 2025

This PR splits the existing stage into "Build" and "Test".

Having separate stages allows a better understanding of the potential gains on each of those part when improving the building process.

Notes:

  • Generated docker compose files are archived even if the tests fail
  • As the publish target includes building the images, no need to call it first when only that one is running on trusted.ci.jenkins.io
  • As there is no "test" phase for multi-arch build, no need to change it

Extracted from:

Testing done

  • CI

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

PS: easier to review without whitespaces, see https://github.com/jenkinsci/docker-agent/pull/1071/files?diff=unified&w=1

@lemeurherve lemeurherve requested a review from a team as a code owner October 5, 2025 22:45
@lemeurherve lemeurherve changed the title chore: split 'Build and Test' stages chore(pipeline): split 'Build and Test' stages Oct 5, 2025
@lemeurherve lemeurherve changed the title chore(pipeline): split 'Build and Test' stages chore(pipeline): split Build and Test stage Oct 5, 2025
@lemeurherve lemeurherve force-pushed the split-build-and-test-stages branch from 89e45b7 to eeb1ede Compare October 5, 2025 22:53
Copy link
Copy Markdown
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@MarkEWaite MarkEWaite merged commit d2da541 into jenkinsci:master Oct 5, 2025
10 checks passed
@lemeurherve lemeurherve deleted the split-build-and-test-stages branch October 5, 2025 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants